Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 22, 2026

Three tests in safe_outputs_mcp_server_defaults.test.cjs were timing out at 10 seconds because the MCP server process needed more time to initialize and respond before assertions ran.

Changes

  • Increased initialization delay: 100ms → 300ms
  • Increased request send delay: 200ms → 600ms
  • Increased assertion delay: 500ms → 1500ms

Applied to:

  • should have optional branch parameter for create_pull_request
  • should have optional branch parameter for push_to_pull_request_branch
  • should return stringified JSON in text content

The tests spawn a Node.js child process running the MCP server and communicate via stdin/stdout. The original delays were insufficient for the server to process tools/list and tools/call requests before the test checked for responses, resulting in undefined values and timeout failures.

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/21259049853/job/61181633543

Debug why the JS tests are failing.

Custom agent used: agentic-workflows
GitHub Agentic Workflows (gh-aw) - Create, debug, and upgrade AI-powered workflows with intelligent prompt routing


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…ponses

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Debug failing JavaScript tests Fix MCP server test timeouts in safe_outputs_mcp_server_defaults Jan 22, 2026
Copilot AI requested a review from pelikhan January 22, 2026 18:26
@pelikhan pelikhan marked this pull request as ready for review January 22, 2026 19:27
@pelikhan pelikhan merged commit bed6eb4 into copilot/update-runtime-manager-mounts Jan 22, 2026
48 checks passed
@pelikhan pelikhan deleted the copilot/debug-js-tests branch January 22, 2026 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants